Skip to content

Peak calling to dev #38

Merged
merged 4 commits into from
Jan 4, 2019
Merged

Peak calling to dev #38

merged 4 commits into from
Jan 4, 2019

Conversation

anastasiia
Copy link
Collaborator

I have just fixed the problem, Jannik had with my output files. His script needs a file that has exactly n-number of columns. As I have not written the columns with bonus info if it was not provided by the original bed file given as input, the script of Jannik could not parse my output file. To make life easier for Jannik, I have just added some small changes, to always have the same number of columns in the output bed file, even if i have no bonus_info from the original bed file. In such a case the last column called bonus_info (as usually) will contain only "."

Also I check for the output directory user could provide as a path to save the output bed file. Just to be sure there are no problems with that.

Copy link
Collaborator

@HendrikSchultheis HendrikSchultheis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your code looks good to me. But I'm not sure whether '.' is a good symbol to indicate an empty field. Your propably want to use something like 'NA' instead. Can you elaborate why you choose '.'?

@anastasiia
Copy link
Collaborator Author

anastasiia commented Jan 4, 2019

Because it is convenient in bed files to use . if there is no information provided.
Besides, the NA is sometimes provided by the input bed file, and this will be written to my output file. So I use the . to show that there is really nothing that I receive from the input file. I used to let the column down, but Jannik needs it.

Copy link
Collaborator

@renewiegandt renewiegandt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good.

@anastasiia anastasiia merged commit 6d0da12 into dev Jan 4, 2019
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants